Skip to content

Upgrade to JDK 17 and JakartaEE 6.0 #138

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from
Closed

Conversation

TareqK
Copy link

@TareqK TareqK commented Dec 11, 2023

Changes

This is a refactor that upgrades the mvc-common library to Jakarta EE6.0 and JDK 17, allowing use on more modern servers and libraries - the tests were also fixed to account for the expiration of tokens. The main changes were to the build and namespaces of imports - with some changes to the tests using regexes.

References

solves #111

Testing

  • This change adds test coverage
  • This change has been tested on the latest version of Java or why not

Checklist

Added Context

This will break the current API as it changes all the required args to another package. Will fail CI check for API Changes - but this is expected and part of the issue.

@TareqK TareqK requested a review from a team as a code owner December 11, 2023 10:21
@TareqK
Copy link
Author

TareqK commented Dec 11, 2023

Further Context : Upgraded only to jdk17 instead of 21 because the oss-plugin didnt run with jdk 21 - and gradle support for jdk 21 is fairly new.

@jimmyjames
Copy link
Contributor

👋 hey @TareqK, thanks for the PR! This is something that we plan to do in a new major version. We may revisit this PR when work on that release begins, but it's not something we can accept right now as it would be a breaking change that would require a new major version release.

@jimmyjames jimmyjames closed this Dec 18, 2023
@nmanandhar
Copy link

hi @jimmyjames

any estimate (months or years) of when the next major version is planned for would be helpful. If it is not planned for the near future, was thinking of maintaining a fork until the fix is released, since it is blocking moving to spring boot 3

@tanya732 tanya732 reopened this Jan 30, 2025
@tanya732 tanya732 requested a review from a team as a code owner January 30, 2025 11:18
@tanya732
Copy link
Contributor

Hi @TareqK,

Thank you for the PR

Can you please sign your commits without which I won't be able to add your commits.

@TareqK
Copy link
Author

TareqK commented Jan 30, 2025

Hi!

Let me actually do some fixed because I see there are conflicts first and I'll get it handled over the weekend hopefully

@mammer42
Copy link

mammer42 commented May 8, 2025

Hello @TareqK, @jimmyjames, @tanya732 ,
are there any plans to merge this MR soon? When do you plan to publish the next major release?
Otherwise, what is the official way to work with auth0 and jdk 17 or 21?

@tanya732
Copy link
Contributor

Hi @mammer42

Thanks for checking in! The multi-Java version testing setup is working smoothly across JDK 8, 11, 17, and 21. We've confirmed that all tests are running successfully on these versions.

As for the major release, it's on our roadmap, and we're targeting an initial version in the coming months. This will include multiple improvements, feature requests, and plugin upgrades, including this PR. We’ll keep everyone updated as we get closer to the release.

@TareqK
Copy link
Author

TareqK commented May 16, 2025

Hello
Apologies for the inactivity. I am going to open a new PR based on a fresh fork, as Equiptal has suspended its engineering activities and the org is no longer functioning on github.

@TareqK
Copy link
Author

TareqK commented May 16, 2025

Hello @tanya732 , im closing this in favour of #156 , which has the same changes and is signed. Lets move the idsucssion there

@TareqK TareqK closed this May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants